cssnode: Optimize listing of classes

Avoid creating the GList of strings when the GQuarks
will just be determined again. Instead have
gtk_style_context_list_classes() do it specifically.

diff --git a/gtk/gtkcssnode.c b/gtk/gtkcssnode.c
index 671613f230e63884114c6dd4370f5f3c0b5f5e8f..9b5db15ad69ff00abf8cbb664ef5d9e24793b55e 100644 (file)
--- a/gtk/gtkcssnode.c
+++ b/gtk/gtkcssnode.c
@@ -1054,16 +1054,15 @@ gtk_css_node_get_junction_sides (GtkCssNode *cssnode)
 static void
 gtk_css_node_clear_classes (GtkCssNode *cssnode)
 {
-  GList *list, *l;
-  
-  list = gtk_css_node_declaration_list_classes (cssnode->decl);
+  const GQuark *classes;
+  guint n_classes, i;
+
+  classes = gtk_css_node_declaration_get_classes (cssnode->decl, &n_classes);
 
-  for (l = list; l; l = l->next)
+  for (i = 0; i < n_classes; ++i)
     {
-      gtk_css_node_remove_class (cssnode, GPOINTER_TO_UINT (l->data));
+      gtk_css_node_remove_class (cssnode, classes[i]);
     }
-
-  g_list_free (list);
 }
 
 void
@@ -1090,22 +1089,20 @@ gtk_css_node_set_classes (GtkCssNode  *cssnode,
 char **
 gtk_css_node_get_classes (GtkCssNode *cssnode)
 {
-  GList *list, *l;
-  GPtrArray *result;
-  
-  list = gtk_css_node_declaration_list_classes (cssnode->decl);
-  result = g_ptr_array_new ();
+  const GQuark *classes;
+  char **result;
+  guint n_classes, i, j;
 
-  for (l = list; l; l = l->next)
+  classes = gtk_css_node_declaration_get_classes (cssnode->decl, &n_classes);
+  result = g_new (char *, n_classes + 1);
+
+  for (i = n_classes, j = 0; i-- > 0; ++j)
     {
-      g_ptr_array_add (result, g_strdup (g_quark_to_string (GPOINTER_TO_UINT (l->data))));
+      result[j] = g_strdup (g_quark_to_string (classes[i]));
     }
 
-  g_ptr_array_add (result, NULL);
-
-  g_list_free (list);
-
-  return (char **) g_ptr_array_free (result, FALSE);
+  result[n_classes] = NULL;
+  return result;
 }
 
 void
@@ -1137,10 +1134,11 @@ gtk_css_node_has_class (GtkCssNode *cssnode,
   return gtk_css_node_declaration_has_class (cssnode->decl, style_class);
 }
 
-GList *
-gtk_css_node_list_classes (GtkCssNode *cssnode)
+const GQuark *
+gtk_css_node_list_classes (GtkCssNode *cssnode,
+                           guint      *n_classes)
 {
-  return gtk_css_node_declaration_list_classes (cssnode->decl);
+  return gtk_css_node_declaration_get_classes (cssnode->decl, n_classes);
 }
 
 void

diff --git a/gtk/gtkcssnodedeclaration.c b/gtk/gtkcssnodedeclaration.c
index 221faef7b0c2979b6b011eb4c2e876f52451cbab..c16189b1fe33bcffecc721b327e633f28facf2a4 100644 (file)
--- a/gtk/gtkcssnodedeclaration.c
+++ b/gtk/gtkcssnodedeclaration.c
@@ -335,22 +335,13 @@ gtk_css_node_declaration_has_class (const GtkCssNodeDeclaration *decl,
     }
 }
 
-GList *
-gtk_css_node_declaration_list_classes (const GtkCssNodeDeclaration *decl)
+const GQuark *
+gtk_css_node_declaration_get_classes (const GtkCssNodeDeclaration *decl,
+                                      guint                       *n_classes)
 {
-  GQuark *classes;
-  GList *result;
-  guint i;
-
-  classes = get_classes (decl);
-  result = NULL;
-
-  for (i = 0; i < decl->n_classes; i++)
-    {
-      result = g_list_prepend (result, GUINT_TO_POINTER (classes[i]));
-    }
+  *n_classes = decl->n_classes;
 
-  return result;
+  return get_classes (decl);
 }
 
 static gboolean
@@ -589,8 +580,7 @@ G_GNUC_END_IGNORE_DEPRECATIONS
   classes = get_classes (decl);
   for (i = 0; i < decl->n_classes; i++)
     {
-      gtk_widget_path_iter_add_class (path, pos,
-                                      g_quark_to_string (classes[i]));
+      gtk_widget_path_iter_add_qclass (path, pos, classes[i]);
     }
 
   /* Set widget state */

diff --git a/gtk/gtkcssnodedeclarationprivate.h b/gtk/gtkcssnodedeclarationprivate.h
index 9e715fe7d214625af4371a339b1c25284a361c9b..51c8c62b1fc38a31431a66eb8bae8180c28eee4c 100644 (file)
--- a/gtk/gtkcssnodedeclarationprivate.h
+++ b/gtk/gtkcssnodedeclarationprivate.h
@@ -47,7 +47,8 @@ gboolean                gtk_css_node_declaration_remove_class           (GtkCssN
                                                                          GQuark                         class_quark);
 gboolean                gtk_css_node_declaration_has_class              (const GtkCssNodeDeclaration   *decl,
                                                                          GQuark                         class_quark);
-GList *                 gtk_css_node_declaration_list_classes           (const GtkCssNodeDeclaration   *decl);
+const GQuark *          gtk_css_node_declaration_get_classes            (const GtkCssNodeDeclaration   *decl,
+                                                                         guint                         *n_classes);
 
 gboolean                gtk_css_node_declaration_add_region             (GtkCssNodeDeclaration        **decl,
                                                                          GQuark                         region_quark,

diff --git a/gtk/gtkcssnodeprivate.h b/gtk/gtkcssnodeprivate.h
index cbed7e8585c4b90f35b5255b9de7a2d7c2a03c6c..9f1dc42b4f2e387df446d4c66f04e1b32365014c 100644 (file)
--- a/gtk/gtkcssnodeprivate.h
+++ b/gtk/gtkcssnodeprivate.h
@@ -134,7 +134,8 @@ void                    gtk_css_node_remove_class       (GtkCssNode            *
                                                          GQuark                 style_class);
 gboolean                gtk_css_node_has_class          (GtkCssNode            *cssnode,
                                                          GQuark                 style_class);
-GList *                 gtk_css_node_list_classes       (GtkCssNode            *cssnode);
+const GQuark *          gtk_css_node_list_classes       (GtkCssNode            *cssnode,
+                                                         guint                 *n_classes);
 void                    gtk_css_node_add_region         (GtkCssNode            *cssnode,
                                                          GQuark                 region,
                                                          GtkRegionFlags         flags);

diff --git a/gtk/gtkstylecontext.c b/gtk/gtkstylecontext.c
index b359612903dc8e2a1484f0ca129a97580ee5b965..643a17b8965a673ceb2d703788d07532a6c0bcbb 100644 (file)
--- a/gtk/gtkstylecontext.c
+++ b/gtk/gtkstylecontext.c
@@ -1391,16 +1391,24 @@ GList *
 gtk_style_context_list_classes (GtkStyleContext *context)
 {
   GtkStyleContextPrivate *priv;
-  GList *classes;
+  GList *classes_list;
+  const GQuark *classes;
+  guint n_classes, i;
+  const gchar *quark_str;
 
   g_return_val_if_fail (GTK_IS_STYLE_CONTEXT (context), NULL);
 
   priv = context->priv;
   
-  classes = gtk_css_node_list_classes (priv->cssnode);
-  quarks_to_strings (classes);
+  classes = gtk_css_node_list_classes (priv->cssnode, &n_classes);
 
-  return classes;
+  for (i = n_classes; i-- > n_classes;)
+    {
+      quark_str = g_quark_to_string (classes[i]);
+      classes_list = g_list_prepend (classes_list, (gchar *) quark_str);
+    }
+
+  return classes_list;
 }
 
 /**

diff --git a/gtk/gtkwidget.c b/gtk/gtkwidget.c
index 76207da2eab6c391bf1f782c3d67a77538d76ba0..26528909ec4539a90955ed601bb96d08d6d2d695 100644 (file)
--- a/gtk/gtkwidget.c
+++ b/gtk/gtkwidget.c
@@ -16333,17 +16333,16 @@ gtk_widget_path_append_for_widget (GtkWidgetPath *path,
 
   if (widget->priv->context)
     {
-      GList *classes, *l;
+      const GQuark *classes;
+      guint n_classes, i;
 
       /* Also add any persistent classes in
        * the style context the widget path
        */
-      classes = gtk_css_node_list_classes (widget->priv->cssnode);
+      classes = gtk_css_node_list_classes (widget->priv->cssnode, &n_classes);
 
-      for (l = classes; l; l = l->next)
-        gtk_widget_path_iter_add_class (path, pos, g_quark_to_string (GPOINTER_TO_UINT (l->data)));
-
-      g_list_free (classes);
+      for (i = n_classes; i-- > 0;)
+        gtk_widget_path_iter_add_qclass (path, pos, classes[i]);
     }
 
   return pos;

diff --git a/gtk/gtkwidgetpath.c b/gtk/gtkwidgetpath.c
index e8a7ac7919696cedc28bf064f28c12ffb34ce175..f83af297a196b4db0940a6cc64b07f2a08902083 100644 (file)
--- a/gtk/gtkwidgetpath.c
+++ b/gtk/gtkwidgetpath.c
@@ -805,20 +805,23 @@ gtk_widget_path_iter_add_class (GtkWidgetPath *path,
                                 gint           pos,
                                 const gchar   *name)
 {
-  GtkPathElement *elem;
-  gboolean added = FALSE;
-  GQuark qname;
-  guint i;
-
   gtk_internal_return_if_fail (path != NULL);
   gtk_internal_return_if_fail (path->elems->len != 0);
   gtk_internal_return_if_fail (name != NULL);
 
-  if (pos < 0 || pos >= path->elems->len)
-    pos = path->elems->len - 1;
+  gtk_widget_path_iter_add_qclass (path, pos, g_quark_from_string (name));
+}
+
+void
+gtk_widget_path_iter_add_qclass (GtkWidgetPath *path,
+                                 gint           pos,
+                                 GQuark         qname)
+{
+  GtkPathElement *elem;
+  gboolean added = FALSE;
+  guint i;
 
   elem = &g_array_index (path->elems, GtkPathElement, pos);
-  qname = g_quark_from_string (name);
 
   if (!elem->classes)
     elem->classes = g_array_new (FALSE, FALSE, sizeof (GQuark));

diff --git a/gtk/gtkwidgetpath.h b/gtk/gtkwidgetpath.h
index f1f7ad7a6a3e788276487b8c9b3c885b5a7c72ee..365c343d796b0a71ba43c73752d00bd08be8feb8 100644 (file)
--- a/gtk/gtkwidgetpath.h
+++ b/gtk/gtkwidgetpath.h
@@ -107,10 +107,18 @@ GDK_AVAILABLE_IN_ALL
 void     gtk_widget_path_iter_add_class     (GtkWidgetPath       *path,
                                              gint                 pos,
                                              const gchar         *name);
+GDK_AVAILABLE_IN_3_18
+void     gtk_widget_path_iter_add_qclass    (GtkWidgetPath       *path,
+                                             gint                 pos,
+                                             GQuark               qname);
 GDK_AVAILABLE_IN_ALL
 void     gtk_widget_path_iter_remove_class  (GtkWidgetPath       *path,
                                              gint                 pos,
                                              const gchar         *name);
+GDK_AVAILABLE_IN_3_18
+void     gtk_widget_path_iter_remove_qclass (GtkWidgetPath       *path,
+                                             gint                 pos,
+                                             GQuark               qname);
 GDK_AVAILABLE_IN_ALL
 void     gtk_widget_path_iter_clear_classes (GtkWidgetPath       *path,
                                              gint                 pos);
@@ -164,6 +172,12 @@ GDK_AVAILABLE_IN_ALL
 gboolean        gtk_widget_path_has_parent (const GtkWidgetPath *path,
                                             GType                type);
 
+/* private API goes here */
+
+void gtk_widget_path_iter_add_qclass       (GtkWidgetPath       *path,
+                                            gint                 pos,
+                                            GQuark               qname);
+
 G_END_DECLS
 
 #endif /* __GTK_WIDGET_PATH_H__ */